Skip to content

Conversation

omarahmed1111
Copy link
Contributor

Enable computeBench to run on sycl with cuda backend via the benchmarking scripts.

@omarahmed1111 omarahmed1111 force-pushed the enable-compute-bench-for-cuda branch from 926d6dd to ac5a3f2 Compare April 18, 2025 11:23
@omarahmed1111 omarahmed1111 marked this pull request as ready for review April 18, 2025 11:23
@omarahmed1111 omarahmed1111 requested a review from a team as a code owner April 18, 2025 11:23
@ianayl
Copy link
Contributor

ianayl commented Apr 21, 2025

Did we not need -DBUILD_SYCL_WITH_CUDA? Additionally, I'm running into issues with compute-benchmarks trying to compile l0: what commands are you using to run main.py to circumvent this (although, there is perhaps a cmake configure setting we're missing here)?

@omarahmed1111
Copy link
Contributor Author

omarahmed1111 commented Apr 22, 2025

Did we not need -DBUILD_SYCL_WITH_CUDA?

I think we do as it it should be enabling cuda backend in sycl targets here

Additionally, I'm running into issues with compute-benchmarks trying to compile l0: what commands are you using to run main.py to circumvent this (although, there is perhaps a cmake configure setting we're missing here)?

python /llvm_path/devops/scripts/benchmarks/main.py /build_path/benchs/bench_cuda --sycl /build_path --output-html --verbose --adapter cuda --timeout 12000

One of the things that I ran into even with l0 is that I need to increase the timeout as compute-bench build takes some time locally.

@omarahmed1111
Copy link
Contributor Author

@intel/llvm-gatekeepers please merge! failures should be unrelated to the PR. Thanks!

@ldrumm ldrumm merged commit 5c514d1 into intel:sycl Apr 23, 2025
25 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants